Skip to content

chore: Update and pin all GHA actions#1222

Merged
mjameswh merged 3 commits intomasterfrom
pin-gha-actions
Apr 18, 2026
Merged

chore: Update and pin all GHA actions#1222
mjameswh merged 3 commits intomasterfrom
pin-gha-actions

Conversation

@mjameswh
Copy link
Copy Markdown
Contributor

@mjameswh mjameswh commented Apr 17, 2026

What changed

  • Bump all GitHub Actions workflows to use latest "safe" releases.
    "Safe" is defined as the latest published release that is at least 2 weeks old (cooldown period).
  • Pin all GHA actions usage to full SHA1, with a version comment.
  • Add explicit GITHUB_TOKEN permissions in workflows

Why

  • Improved security.

@mjameswh mjameswh requested a review from a team as a code owner April 17, 2026 23:48
Comment on lines 15 to 26
timeout-minutes: 20
runs-on: ubuntu-latest-16-cores
steps:
- uses: actions/checkout@v4
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
with:
submodules: recursive
- uses: dtolnay/rust-toolchain@stable
- uses: dtolnay/rust-toolchain@29eef336d9b2848a0b548edc03f92a220660cdb8 # stable
- name: Install protoc
uses: arduino/setup-protoc@v3
uses: arduino/setup-protoc@c65c819552d16ad3c9b72d9dfd5ba5237b9c906b # v3
with:
# TODO: Upgrade proto once https://github.com/arduino/setup-protoc/issues/99 is fixed
version: "23.x"
Copy link
Copy Markdown

@semgrep-managed-scans semgrep-managed-scans Bot Apr 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No explicit GITHUB_TOKEN permissions found at the workflow or job level. Add a permissions: block at the workflow root (applies to all jobs) or per job with least privilege (e.g., contents: read and only specific writes like pull-requests: write if needed).

🥳 Fixed in commit b6a450d 🥳

Semgrep rule temporal.security.gha.missing-explicit-permissions: set least-privilege permissions blocks.
Align with Semgrep missing-explicit-permissions (job-scoped) expectations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants